Skip to content

Conversation

michaelwoerister
Copy link
Member

Make vtables encodable so we can cache the trans_fulfill_obligation query at some point.

r? @eddyb

const VTABLE_BUILTIN: u8 = 4;
const VTABLE_CLOSURE: u8 = 5;
const VTABLE_FN_POINTER: u8 = 6;
const VTABLE_GENERATOR: u8 = 7;
Copy link
Member

@eddyb eddyb Nov 30, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, since when do we do this manually?!

@eddyb
Copy link
Member

eddyb commented Nov 30, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Nov 30, 2017

📌 Commit 780f4ed has been approved by eddyb

@kennytm kennytm added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 30, 2017
arielb1 pushed a commit to arielb1/rust that referenced this pull request Nov 30, 2017
…r=eddyb

incr.comp.: Make traits::VTable encodable and decodable.

Make vtables encodable so we can cache the `trans_fulfill_obligation` query at some point.

r? @eddyb
arielb1 pushed a commit to arielb1/rust that referenced this pull request Nov 30, 2017
…r=eddyb

incr.comp.: Make traits::VTable encodable and decodable.

Make vtables encodable so we can cache the `trans_fulfill_obligation` query at some point.

r? @eddyb
arielb1 pushed a commit to arielb1/rust that referenced this pull request Nov 30, 2017
…r=eddyb

incr.comp.: Make traits::VTable encodable and decodable.

Make vtables encodable so we can cache the `trans_fulfill_obligation` query at some point.

r? @eddyb
kennytm added a commit to kennytm/rust that referenced this pull request Dec 1, 2017
…r=eddyb

incr.comp.: Make traits::VTable encodable and decodable.

Make vtables encodable so we can cache the `trans_fulfill_obligation` query at some point.

r? @eddyb
bors added a commit that referenced this pull request Dec 1, 2017
Rollup of 13 pull requests

- Successful merges: #45880, #46280, #46373, #46376, #46385, #46386, #46387, #46392, #46400, #46401, #46405, #46412, #46421
- Failed merges:
@bors bors merged commit 780f4ed into rust-lang:master Dec 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants